Skip to content

fix: Hot/cold combination of a wallet doesn't work smooth - #3000

Merged
devchenyan merged 15 commits into
developfrom
fix-288
Jan 29, 2024
Merged

fix: Hot/cold combination of a wallet doesn't work smooth#3000
devchenyan merged 15 commits into
developfrom
fix-288

Conversation

@devchenyan

@devchenyan devchenyan commented Dec 22, 2023

Copy link
Copy Markdown
Contributor

issue: Magickbase/neuron-public-issues#288

Screen-2024-01-01-235512.mp4

@devchenyan
devchenyan requested review from Keith-CY, WhiteMinds and yanguoyu and removed request for Keith-CY and yanguoyu December 24, 2023 01:55
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/locales/en.json Outdated
Comment thread packages/neuron-ui/src/locales/en.json Outdated
Comment thread packages/neuron-wallet/src/controllers/app/menu.ts Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
@Keith-CY

Copy link
Copy Markdown
Contributor

Any update on the conversations above?

Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-wallet/src/controllers/offline-sign.ts Outdated
@nervosnetwork nervosnetwork deleted a comment from yanguoyu Jan 11, 2024
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/locales/en.json
Comment thread packages/neuron-ui/src/locales/en.json Outdated
Comment thread packages/neuron-ui/src/services/remote/offline.ts Outdated
Comment thread packages/neuron-wallet/src/controllers/app/menu.ts Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-ui/src/components/BroadcastTransaction/index.tsx Outdated
Comment thread packages/neuron-wallet/src/controllers/offline-sign.ts Outdated
@devchenyan
devchenyan requested a review from yanguoyu January 14, 2024 02:59
@devchenyan

devchenyan commented Jan 14, 2024

Copy link
Copy Markdown
Contributor Author

/package
Packaging for test is done in 7518149207. @devchenyan

@silySuper

silySuper commented Jan 19, 2024

Copy link
Copy Markdown

/package
Packaging for test is done in 7581685497. @silySuper

@silySuper

silySuper commented Jan 29, 2024

Copy link
Copy Markdown

/package
Packaging for test is done in 7690224343. @silySuper

@silySuper

Copy link
Copy Markdown

1.Why not calls unsigned transaction here?
截屏2024-01-29 15 25 18

@devchenyan

Copy link
Copy Markdown
Contributor Author

1.Why not calls unsigned transaction here? 截屏2024-01-29 15 25 18

This was suggested to be revised in the above comment.
#3000 (comment)

@devchenyan
devchenyan added this pull request to the merge queue Jan 29, 2024
Merged via the queue into develop with commit 2809428 Jan 29, 2024
@devchenyan
devchenyan deleted the fix-288 branch January 29, 2024 09:44
yanguoyu pushed a commit to yanguoyu/neuron that referenced this pull request Mar 14, 2024
…ork#3000)

* fix: issue 288

* fix: broadcastTransaction

* fix: BroadcastTransaction

* fix: French

* fix: broadcastTransactionOnly

* fix: comments

* fix: broadcastSignedTransaction

* fix

* feat: remove Wallet field

---------

Co-authored-by: Chen Yu <keithwhisper@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants